Skip to content

Conversation

@luancazarine
Copy link
Collaborator

@luancazarine luancazarine commented Jun 18, 2025

Resolves #17079

Summary by CodeRabbit

  • New Features

    • Added a new event source to emit notifications when a voice call is recorded, including detailed call metadata.
    • Introduced a sample event for testing the voice call recorded source.
  • Improvements

    • Updated event source names to indicate instant event delivery for inbound messages, internal notes, outbound messages, phone call events, and ticket label additions.
    • Incremented version numbers for all actions and sources to reflect the latest updates.
    • Upgraded package and dependency versions for improved compatibility.

@luancazarine luancazarine linked an issue Jun 18, 2025 that may be closed by this pull request
@vercel
Copy link

vercel bot commented Jun 18, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Skipped Deployments
Name Status Preview Comments Updated (UTC)
docs-v2 ⬜️ Ignored (Inspect) Visit Preview Jun 18, 2025 8:08pm
pipedream-docs ⬜️ Ignored (Inspect) Jun 18, 2025 8:08pm
pipedream-docs-redirect-do-not-edit ⬜️ Ignored (Inspect) Jun 18, 2025 8:08pm

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jun 18, 2025

Walkthrough

This update introduces a new Trengo source trigger for "VOICE_CALL_RECORDED" events, including a test event and event metadata logic. Additionally, several Trengo actions and sources have their version numbers incremented, and minor metadata and import order adjustments are made. No other functional changes are present.

Changes

File(s) Change Summary
components/trengo/sources/voice-call-recorded/voice-call-recorded.mjs,
components/trengo/sources/voice-call-recorded/test-event.mjs
Added new "VOICE_CALL_RECORDED" source trigger and test event export
components/trengo/actions/create-contact/create-contact.mjs
components/trengo/actions/find-contacts/find-contacts.mjs
components/trengo/actions/log-a-voice-call/log-a-voice-call.mjs
components/trengo/actions/send-a-message/send-a-message.mjs
components/trengo/actions/send-a-team-chat-message/send-a-team-chat-message.mjs
components/trengo/actions/send-a-whatsapp-message-template/send-a-whatsapp-message-template.mjs
Incremented action version numbers and adjusted some import orders
components/trengo/sources/new-inbound-message/new-inbound-message.mjs
components/trengo/sources/new-internal-note/new-internal-note.mjs
components/trengo/sources/new-outbound-message/new-outbound-message.mjs
components/trengo/sources/phone-call-ended/phone-call-ended.mjs
components/trengo/sources/phone-call-missed/phone-call-missed.mjs
components/trengo/sources/phone-call-started/phone-call-started.mjs
components/trengo/sources/ticket-label-added/ticket-label-added.mjs
Updated source metadata: incremented version numbers and renamed event names to include "(Instant)"
components/trengo/package.json Updated package version and dependency version

Sequence Diagram(s)

sequenceDiagram
    participant Trengo as Trengo Platform
    participant Source as VOICE_CALL_RECORDED Source
    participant User as User

    Trengo->>Source: Emits VOICE_CALL_RECORDED event
    Source->>Source: getMeta(event) extracts metadata
    Source->>User: Emits processed event with summary and metadata
Loading

Assessment against linked issues

Objective Addressed Explanation
Add a new trigger for Trengo - VOICE_CALL_RECORDED (#17079)

Assessment against linked issues: Out-of-scope changes

Code Change Explanation
Version bumps and metadata renaming in unrelated Trengo actions and sources (various files) These changes are not required for the addition of the VOICE_CALL_RECORDED trigger as described.
Package version and dependency update in components/trengo/package.json Not necessary for the specific trigger addition; may be general maintenance or unrelated improvement.

Poem

In the garden of code where new triggers are grown,
A voice call recorded now stands on its own.
With versions all bumped and sources made neat,
The bunnies of Trengo hop on their feet.
Events now arrive, so crisp and so clear—
Hooray for new features, let’s all give a cheer!
🐇📞✨

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

components/trengo/sources/voice-call-recorded/voice-call-recorded.mjs

Oops! Something went wrong! :(

ESLint: 8.57.1

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'jsonc-eslint-parser' imported from /eslint.config.mjs
at Object.getPackageJSONURL (node:internal/modules/package_json_reader:255:9)
at packageResolve (node:internal/modules/esm/resolve:767:81)
at moduleResolve (node:internal/modules/esm/resolve:853:18)
at defaultResolve (node:internal/modules/esm/resolve:983:11)
at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:801:12)
at #cachedDefaultResolve (node:internal/modules/esm/loader:725:25)
at ModuleLoader.resolve (node:internal/modules/esm/loader:708:38)
at ModuleLoader.getModuleJobForImport (node:internal/modules/esm/loader:309:38)
at #link (node:internal/modules/esm/module_job:201:49)


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 18a13ea and 7a6190a.

📒 Files selected for processing (1)
  • components/trengo/sources/voice-call-recorded/voice-call-recorded.mjs (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • components/trengo/sources/voice-call-recorded/voice-call-recorded.mjs
⏰ Context from checks skipped due to timeout of 90000ms (4)
  • GitHub Check: pnpm publish
  • GitHub Check: Lint Code Base
  • GitHub Check: Publish TypeScript components
  • GitHub Check: Verify TypeScript components
✨ Finishing Touches
  • 📝 Generate Docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🔭 Outside diff range comments (1)
components/trengo/sources/new-internal-note/new-internal-note.mjs (1)

19-19: Fix summary text typo
The summary currently reads “New ticket label added event…” but this is the Internal Note source. Update to something like:

- summary: `New ticket label added event: ${event?.body?.message}`,
+ summary: `New internal note event: ${event?.body?.message}`,
🧹 Nitpick comments (2)
components/trengo/sources/new-outbound-message/new-outbound-message.mjs (1)

19-19: Typo in summary string
“outbund” should be “outbound”. Recommend:

- summary: `New outbund message event: ${event?.body?.message}`,
+ summary: `New outbound message event: ${event?.body?.message}`,
components/trengo/sources/phone-call-missed/phone-call-missed.mjs (1)

6-6: Grammar in description
Change “when an phone call missed” to “when a phone call is missed” for clarity:

- description: "Emit new events when an phone call missed.…",
+ description: "Emit new events when a phone call is missed.…",
📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0a7bc29 and 18a13ea.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (16)
  • components/trengo/actions/create-contact/create-contact.mjs (1 hunks)
  • components/trengo/actions/find-contacts/find-contacts.mjs (1 hunks)
  • components/trengo/actions/log-a-voice-call/log-a-voice-call.mjs (1 hunks)
  • components/trengo/actions/send-a-message/send-a-message.mjs (1 hunks)
  • components/trengo/actions/send-a-team-chat-message/send-a-team-chat-message.mjs (1 hunks)
  • components/trengo/actions/send-a-whatsapp-message-template/send-a-whatsapp-message-template.mjs (1 hunks)
  • components/trengo/package.json (2 hunks)
  • components/trengo/sources/new-inbound-message/new-inbound-message.mjs (1 hunks)
  • components/trengo/sources/new-internal-note/new-internal-note.mjs (1 hunks)
  • components/trengo/sources/new-outbound-message/new-outbound-message.mjs (1 hunks)
  • components/trengo/sources/phone-call-ended/phone-call-ended.mjs (1 hunks)
  • components/trengo/sources/phone-call-missed/phone-call-missed.mjs (1 hunks)
  • components/trengo/sources/phone-call-started/phone-call-started.mjs (1 hunks)
  • components/trengo/sources/ticket-label-added/ticket-label-added.mjs (1 hunks)
  • components/trengo/sources/voice-call-recorded/test-event.mjs (1 hunks)
  • components/trengo/sources/voice-call-recorded/voice-call-recorded.mjs (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (4)
  • GitHub Check: pnpm publish
  • GitHub Check: Publish TypeScript components
  • GitHub Check: Verify TypeScript components
  • GitHub Check: Lint Code Base
🔇 Additional comments (23)
components/trengo/package.json (2)

3-3: Bump component version to 0.1.0.
Minor version bump aligns with the introduction of new features and the tracked issue resolution.


13-13: Upgrade @pipedream/platform dependency to ^3.1.0.
This is a major version bump—please verify that all existing actions and sources remain compatible with the updated platform API.

components/trengo/actions/log-a-voice-call/log-a-voice-call.mjs (1)

6-6: Increment action version to 0.0.2.
Version bump is consistent with other Trengo actions in this PR. No functional changes detected.

components/trengo/actions/create-contact/create-contact.mjs (1)

6-6: Increment action version to 0.0.2.
This aligns with the coordinated version bump across Trengo actions.

components/trengo/actions/send-a-message/send-a-message.mjs (1)

6-6: Increment action version to 0.0.2.
Consistent version update; no logic changes.

components/trengo/sources/phone-call-started/phone-call-started.mjs (1)

5-7: Suffix event name with “(Instant)” and bump version to 0.0.2.
Display name change and version increment are in line with other source updates in this PR.

components/trengo/sources/new-internal-note/new-internal-note.mjs (2)

5-5: Append “(Instant)” suffix to source name
Name updated to “New Internal Note Event (Instant)” for consistency with other Trengo sources.


7-7: Bump version to “0.0.2”
Version increment aligns with coordinated metadata updates across all Trengo components.

components/trengo/sources/ticket-label-added/ticket-label-added.mjs (2)

5-5: Append “(Instant)” suffix to source name
Consistent naming applied: “New Ticket Label Added Event (Instant)”.


7-7: Bump version to “0.0.2”
Matches the version bump strategy used across all Trengo sources in this PR.

components/trengo/sources/new-outbound-message/new-outbound-message.mjs (2)

5-5: Append “(Instant)” suffix to source name
“New Outbound Message Event (Instant)” aligns with other event definitions.


7-7: Bump version to “0.0.2”
Version update is in lockstep with related source components.

components/trengo/sources/phone-call-missed/phone-call-missed.mjs (2)

5-5: Append “(Instant)” suffix to source name
Updated to “New Phone Call Missed Event (Instant)” for consistency.


7-7: Bump version to “0.0.2”
Version bump aligns with mass update across Trengo sources.

components/trengo/sources/phone-call-ended/phone-call-ended.mjs (2)

5-5: Append “(Instant)” suffix to source name
Consistent suffix applied: “New Phone Call Ended Event (Instant)”.


7-7: Bump version to “0.0.2”
Matches version updates across other Trengo components.

components/trengo/sources/new-inbound-message/new-inbound-message.mjs (1)

5-5: LGTM! Consistent metadata updates.

The name clarification with "(Instant)" and version increment align well with the PR's broader component updates.

Also applies to: 7-7

components/trengo/sources/voice-call-recorded/voice-call-recorded.mjs (1)

1-24: LGTM! Well-structured new voice call recorded trigger.

The implementation follows established Trengo source patterns and correctly extracts event metadata. The summary format is informative and the event type is clearly defined.

components/trengo/sources/voice-call-recorded/test-event.mjs (1)

1-13: LGTM! Comprehensive test event data.

The test event provides all necessary fields with realistic data that aligns with the voice call recorded source component's metadata extraction logic.

components/trengo/actions/send-a-team-chat-message/send-a-team-chat-message.mjs (1)

2-2: LGTM! Consistent maintenance updates.

The import reordering and version increment are part of the coordinated component updates across the PR.

Also applies to: 7-7

components/trengo/actions/send-a-whatsapp-message-template/send-a-whatsapp-message-template.mjs (1)

2-2: LGTM! Consistent maintenance updates.

The import reordering and version increment align with the coordinated component updates throughout this PR.

Also applies to: 7-7

components/trengo/actions/find-contacts/find-contacts.mjs (2)

2-2: LGTM! Import reordering for consistency.

The import statement reordering doesn't affect functionality and likely improves code organization consistency across the project.


7-7: LGTM! Appropriate version increment.

The patch version bump from 0.0.1 to 0.0.2 is appropriate for the minor changes made and aligns with the coordinated version updates across Trengo components.

Copy link
Collaborator

@michelle0927 michelle0927 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@luancazarine luancazarine merged commit d223e5f into master Jun 19, 2025
11 checks passed
@luancazarine luancazarine deleted the 17079-trigger-trengo---voice_call_recorded branch June 19, 2025 14:58
joy-chanboop pushed a commit to joy-chanboop/pipedream that referenced this pull request Jun 21, 2025
* [TRIGGER] Trengo - VOICE_CALL_RECORDED PipedreamHQ#17079
Sources
 - Voice Call Recorded

* update pnpm-lock.yaml

* Update components/trengo/sources/voice-call-recorded/voice-call-recorded.mjs

---------

Co-authored-by: michelle0927 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[TRIGGER] Trengo - VOICE_CALL_RECORDED

3 participants